Skip to content

fix(providers): show a No Auth provider's real requirement on its banner - #15363

Merged
diegosouzapw merged 1 commit into
cheaper-inference:release/v3.8.52from
HouMinXi:fix/noauth-banner
Oct 6, 2026
Merged

diegosouzapw merged 1 commit into
cheaper-inference:release/v3.8.52from
HouMinXi:fix/noauth-banner

Conversation

@HouMinXi

@HouMinXi HouMinXi commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • The No Auth section showed every provider the same banner, saying it was ready to use with no signup and no API key. That is wrong for providers that need a local component, a browser binary or a matching client.
  • The real requirement was already written in each provider's notice in noauth.ts. The banner now appends that notice when one exists, and a provider without one keeps the original banner.
  • No new copy was written.

Related Issues

Validation

  • Change type: other (dashboard banner)
  • Focused tests: node --import tsx/esm --test tests/unit/noauth-banner-requirement.test.ts (3 pass)
  • eslint on the changed files, with the repo suppression file
  • Reconciled with release/v3.8.52 @ 23a1148486
  • New test covers the change
  • Defect injection: removing the two resolver calls turns the cloudflare and opencode cases red while the duckduckgo case stays green, and the change was restored

Tests Added Or Updated

  • tests/unit/noauth-banner-requirement.test.ts (added)

Coverage Notes

  • The test checks a provider with a notice, a provider without one, and the account card path.

Reviewer Notes

  • authHint is not used. Only notice.text is appended, because that is the field that states a hard prerequisite.

@HouMinXi
HouMinXi requested a review from diegosouzapw as a code owner October 2, 2026 17:15
The No Auth section told every provider it was ready to use with no
signup and no API key. Several of them need a local component, a browser
binary or a matching client, and that requirement was already written in
each provider's notice. The banner now appends that notice when one
exists and stays unchanged when it does not.

Signed-off-by: Minxi Hou <houminxi@gmail.com>
@diegosouzapw
diegosouzapw merged commit a45b818 into cheaper-inference:release/v3.8.52 Oct 6, 2026
42 of 51 checks passed
@HouMinXi
HouMinXi deleted the fix/noauth-banner branch October 28, 2026 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants